]> dgit.raspbian.org Git - gnome-shell.git/commit
workspacesView: Avoid setting invalid geometries on views
authorJonas Dreßler <verdre@v0yd.nl>
Wed, 20 May 2020 10:05:04 +0000 (12:05 +0200)
committerFlorian Müllner <florian.muellner@gmail.com>
Sun, 14 Jun 2020 14:44:10 +0000 (14:44 +0000)
commitbaeb4079ee2f005e226fd85e4f9e88e234ffc93c
tree6815edaf11f8c325796f304ffbd31047e030b511
parentf91f9801b4c19ff595d39b7573e58019ee2b37e7
workspacesView: Avoid setting invalid geometries on views

The fullGeometry and the actualGeometry of the WorkspacesDisplay are set
from the allocation of the overviews ControlsManager and the
WorkspacesDisplay, that means they're only valid after those actors got
their allocations during Clutters allocation cycle.

Since WorkspacesDisplay._updateWorkspacesViews() is already called while
showing/mapping the WorkspacesDisplay, that allocation cycle didn't
happen yet and we end up either setting the geometries of the views to
null (in case of the fullGeometry) or to something wrong (a 0-sized
allocation in case of the actualGeometry).

So avoid setting invalid geometries on the views by initializing both
the fullGeometry and the actualGeometry to null, and then only updating
the geometries of the views after they're set to a correct value.

Note that this means we won't correctly animate the overview the first
time we open it since the animation depends on the geometries being set,
but is being started from show(), which means no allocations have
happened yet. In practice this introduces no regression though since
before this change we simply used incorrect geometries (see the 0-sized
allocation mentioned above) on the initial opening and the animation
didn't work either.

https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1119
js/ui/workspacesView.js